Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature epo 614 #116

Merged
merged 2 commits into from
Oct 27, 2022
Merged

Feature epo 614 #116

merged 2 commits into from
Oct 27, 2022

Conversation

dbd311
Copy link
Contributor

@dbd311 dbd311 commented Oct 27, 2022

No description provided.

Copy link
Collaborator

@costezki costezki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

look good.

Please change the input location of XMIs from "test" folder to something else.

@@ -20,8 +20,7 @@
<xd:p>This module defines project level variables and parameters</xd:p>
</xd:desc>
</xd:doc>
<!-- List of xmi files to be merged into a single xmi file-->
<xsl:variable name="xmiFileList" select="collection('../../test/test-multi-xmi?select=*.xmi')"/>

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indeed we do not need this

@@ -14,15 +14,15 @@
<prefix name="skos" value="http://www.w3.org/2004/02/skos/core#"/>

<prefix name="epo" value="http://data.europa.eu/a4g/ontology#"/>
<prefix name="epo-not" value="http://data.europa.eu/a4g/ontology#"/>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good input for the moment.

I fear we will need other namespace definitions too, at least for eOrdering, eFulfillment.
you may as well add them now: "epo-cat", "epo-ful" (same base URI as above)

@@ -16,6 +16,9 @@

<xsl:import href="../../config/config-proxy.xsl"/>

<!-- List of xmi files to be merged into a single xmi file-->
<xsl:variable name="xmiFileList" select="collection('../../test/test-multi-xmi?select=*.xml')"/>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do we want to take XMI files from a "test" location?
This shall not be in the master branch. We need a proper mechanism to indicate where is the input location.

@dbd311 dbd311 merged commit 3f14ebc into master Oct 27, 2022
@dbd311 dbd311 deleted the feature-epo-614 branch October 27, 2022 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants